fix(asr): 修复百炼 ASR 连通性校验死锁并透出详细错误信息#848
Merged
Merged
Conversation
Contributor
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
lvsao
changed the title
Collaborator
|
批准合并 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
摘要
修复阿里云百炼（Bailian/DashScope）实时 ASR 供应商点击验证始终显示「操作失败」的问题。
修复
后端死锁（bailian.rs）： inish_error() 未调用 self.task_started.notify_waiters()。当 WebSocket 连接或 DashScope 会话建立过程中发生错误时，send_last_frame() 中等待 ask_started 的异步 Notify 无法收到唤醒信号，导致 5 秒超时，最终抛出 FinalResultTimeout，掩盖了真实的 DashScope 错误原因。
前端错误吞没（ProvidersSection.tsx）：providerErrorMessage() 中所有未匹配特定前缀的错误消息均降级为 ('common.operationFailed')（「操作失败」）。现已放行 ask failed: / connection failed: / send failed: 等后端原始错误文案，使用户能看到具体的失败原因。
兼容
测试计划